Skip to content

[CDAP-21093] Add errorCodeType, errorCode & supportedDocURL in exception (MySQL/CloudSQLMySQLErrorDetailsProvider)#530

Merged
psainics merged 1 commit intodata-integrations:developfrom
cloudsufi:patch/errorCodeTypeExtention
Dec 18, 2024
Merged

[CDAP-21093] Add errorCodeType, errorCode & supportedDocURL in exception (MySQL/CloudSQLMySQLErrorDetailsProvider)#530
psainics merged 1 commit intodata-integrations:developfrom
cloudsufi:patch/errorCodeTypeExtention

Conversation

@psainics
Copy link
Contributor

@psainics psainics commented Dec 17, 2024

Add errorCodeType, errorCode & supportedDocURL in exception

Jira : CDAP-21093

Description

Adding errorCodeType, errorCode & supportedDocURL in exception for mysql, and command DBErrorDetailsProvider class

Code change

  • Modified DBErrorDetailsProvider.java
  • Modified DBRecord.java
  • Modified DBUtils.java
  • Modified MysqlErrorDetailsProvider.java

Test

image
image

image

@psainics psainics added the build label Dec 17, 2024
@psainics psainics self-assigned this Dec 17, 2024
@psainics psainics changed the title [CDAP-21093] Add errorCodeType, errorCode & supportedDocURL in exception [CDAP-21093] Add errorCodeType, errorCode & supportedDocURL in exception (MySQL) Dec 17, 2024
@psainics psainics requested a review from AnkitCLI December 17, 2024 04:14
@psainics psainics marked this pull request as ready for review December 17, 2024 04:49
return ErrorUtils.getProgramFailureException(new ErrorCategory(ErrorCategory.ErrorCategoryEnum.PLUGIN),
errorMessage, errorMessageWithDetails, getErrorTypeFromErrorCode(errorCode), false, e);
errorMessage, errorMessageWithDetails, getErrorTypeFromErrorCode(errorCode), false, ErrorCodeType.SQLSTATE,
sqlState, DBUtils.MYSQL_SUPPORTED_DOC_URL, e);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The supported doc url should come from child class, we cannot hardcoded mysql.

Copy link
Contributor Author

@psainics psainics Dec 17, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤦 typo, Fixed !


Tested Again

image

@psainics psainics force-pushed the patch/errorCodeTypeExtention branch from 1d0fb5c to 98792ac Compare December 17, 2024 05:03
@psainics psainics force-pushed the patch/errorCodeTypeExtention branch from 98792ac to 5c958cb Compare December 17, 2024 22:09
@psainics
Copy link
Contributor Author

Merged #526 , Doc changes for CloudSQLMySQLErrorDetailsProvider are included in this PR.

@psainics psainics changed the title [CDAP-21093] Add errorCodeType, errorCode & supportedDocURL in exception (MySQL) [CDAP-21093] Add errorCodeType, errorCode & supportedDocURL in exception (MySQL/CloudSQLMySQLErrorDetailsProvider) Dec 18, 2024
@psainics psainics force-pushed the patch/errorCodeTypeExtention branch from 5c958cb to 5937192 Compare December 18, 2024 02:39
@psainics psainics merged commit 0ef198a into data-integrations:develop Dec 18, 2024
9 checks passed
@psainics psainics deleted the patch/errorCodeTypeExtention branch December 23, 2024 07:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants